Skip to content

Añadidos ganchos a la vista twig del dashboard#1891

Merged
NeoRazorX merged 1 commit intoNeoRazorX:masterfrom
daniel89fg:ganchos-dashboard
Feb 24, 2026
Merged

Añadidos ganchos a la vista twig del dashboard#1891
NeoRazorX merged 1 commit intoNeoRazorX:masterfrom
daniel89fg:ganchos-dashboard

Conversation

@daniel89fg
Copy link
Copy Markdown
Contributor

El controladora Dashboard tiene pipes(), pero la vista twit no tenía ganchos, obligando a usar herencia en vez de extensiones.

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

Copilot AI review requested due to automatic review settings February 24, 2026 12:19
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds extension hooks to the Dashboard Twig template to enable plugins to inject content without requiring template inheritance. The Dashboard controller already supports extensibility through pipe() method calls, but the corresponding Twig template lacked the necessary hooks.

Changes:

  • Added six extension hooks (Before/After pairs) around three template blocks: sectionLinks, sectionStats, and sections
  • Added documentation comments to clarify the purpose of each section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@NeoRazorX NeoRazorX merged commit a1636e4 into NeoRazorX:master Feb 24, 2026
25 of 26 checks passed
@daniel89fg daniel89fg deleted the ganchos-dashboard branch February 25, 2026 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants